Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Account layout template adjustments #2623

Merged
merged 1 commit into from
Oct 28, 2021
Merged

Conversation

danacotoran
Copy link
Contributor

Implement some adjustments to the layout template for accounts to ensure
a smooth transition to DI auth.

Remove the Account product name from the header, and ensure the GOVUK
link leads back to the GOVUK homepage so users don't get stuck in the
account zone.

Remove footer navigation links in favour or the more stripped down
version of the footer. We want the footer to match what DI have
implemented (it could look weird if the footer dramatically changes when
someone navigates from one account page to another)

https://trello.com/c/vIjbnGSj/1087-account-pages-tidy-up-for-launch

Related PR which implements similar minor changes to the account layout in the components gem: alphagov/govuk_publishing_components#2380

@govuk-ci govuk-ci temporarily deployed to govuk-static-account-te-tuszbh October 25, 2021 16:37 Inactive
@govuk-ci govuk-ci temporarily deployed to govuk-static-account-te-tuszbh October 26, 2021 14:49 Inactive
@danacotoran danacotoran changed the title Account layout template adjustments [DO NOT MERGE] Account layout template adjustments Oct 27, 2021
@govuk-ci govuk-ci had a problem deploying to govuk-static-account-te-tuszbh October 27, 2021 10:12 Failure
@danacotoran danacotoran force-pushed the account-template-adjustments branch from a03aad4 to 30ea689 Compare October 27, 2021 10:17
@govuk-ci govuk-ci had a problem deploying to govuk-static-account-te-tuszbh October 27, 2021 10:17 Failure
@danacotoran danacotoran force-pushed the account-template-adjustments branch 2 times, most recently from c4b902b to d8c2b0a Compare October 27, 2021 10:28
@govuk-ci govuk-ci temporarily deployed to govuk-static-account-te-tuszbh October 27, 2021 10:28 Inactive
Implement some adjustments to the layout template for accounts to ensure
a smooth transition to DI auth.

Remove the Account product name from the header, and ensure the GOVUK
link leads back to the GOVUK homepage so users don't get stuck in the
account zone.

Remove footer navigation links in favour or the more stripped down
version of the footer. We want the footer to match what DI have
implemented (it could look weird if the footer dramatically changes when
someone navigates from one account page to another)
@danacotoran danacotoran force-pushed the account-template-adjustments branch from d8c2b0a to 0233c2b Compare October 27, 2021 10:32
@govuk-ci govuk-ci temporarily deployed to govuk-static-account-te-tuszbh October 27, 2021 10:32 Inactive
@govuk-ci govuk-ci had a problem deploying to govuk-static-account-te-tuszbh October 27, 2021 10:47 Failure
@danacotoran danacotoran force-pushed the account-template-adjustments branch from 1e5d4e5 to ff12b9b Compare October 27, 2021 10:49
@govuk-ci govuk-ci had a problem deploying to govuk-static-account-te-tuszbh October 27, 2021 10:49 Failure
@danacotoran danacotoran force-pushed the account-template-adjustments branch from ff12b9b to aab9f52 Compare October 27, 2021 10:50
@govuk-ci govuk-ci temporarily deployed to govuk-static-account-te-tuszbh October 27, 2021 10:51 Inactive
@danacotoran danacotoran force-pushed the account-template-adjustments branch from aab9f52 to 0233c2b Compare October 28, 2021 08:38
@govuk-ci govuk-ci temporarily deployed to govuk-static-account-te-tuszbh October 28, 2021 08:38 Inactive
@danacotoran danacotoran changed the title [DO NOT MERGE] Account layout template adjustments Account layout template adjustments Oct 28, 2021
@danacotoran danacotoran merged commit c4839f4 into main Oct 28, 2021
@danacotoran danacotoran deleted the account-template-adjustments branch October 28, 2021 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants